Can one of the admins verify this patch?

@swift-server-bot test this please

ah meh, too early

Thanks for the PR; would you mind fixing the #if the same way as we did here: apple/swift-service-context@46faa2a ? Thanks!

Thanks for the review and fix @ktoso! Are you going to draft a release for the baggage package so we can change the dependency version in this PR?

@swift-server-bot add to allowlist

Sure we can tag if you want -- are you using this package already? In what capacity / situation?

I'm currently working on my bachelor's thesis in the realm of distributed tracing. At the chair, we are developing a declarative framework for building web services in Swift, called Apodini. My focus is in part to add distributed tracing capabilities, which is where swift-distributed-tracing comes in 🚀

That looks quite nice :-) Well done and good luck with the thesis!
Feel free to reach out here or on the https://forums.swift.org with any questions, happy to support.

Thank you!

In the current state of main after merging this PR, the package doesn't build on macOS, because the Baggage.current TaskLocal property is of course only available on macOS 12 in the 0.2.x release. I think the CI checks only run on Linux, do they?

Yeah, we only have Linux CI, however with the bump of the dependency to 0.2.2 things build fine AFAIK (I built locally, seems good).

I just tagged 0.2.2 here as well, feel free to try.
Sorry I'm a bit in a rush with those things, hope I got it right :-)

FYI @moritzsternemann

Looks good to me as well, thank you!